libostree: Use ostree_repo_create() rather than spawning subprocess
authorColin Walters <walters@verbum.org>
Mon, 16 Sep 2013 12:52:16 +0000 (08:52 -0400)
committerColin Walters <walters@verbum.org>
Mon, 16 Sep 2013 12:52:16 +0000 (08:52 -0400)
We have API now, yay!

src/libostree/ostree-sysroot.c

index 20b266d04977cd3a7cd9f5b671d7c14f6edd029c..e7bc34f4e6511522a784af3bcfdc88108d01ce65 100644 (file)
@@ -206,12 +206,11 @@ ostree_sysroot_ensure_initialized (OstreeSysroot  *self,
   gboolean ret = FALSE;
   gs_unref_object GFile *dir = NULL;
   gs_unref_object GFile *ostree_dir = NULL;
+  gs_unref_object GFile *repo_dir = NULL;
 
   ostree_dir = g_file_get_child (self->path, "ostree");
-
-  g_clear_object (&dir);
-  dir = g_file_get_child (ostree_dir, "repo");
-  if (!gs_file_ensure_directory (dir, TRUE, cancellable, error))
+  repo_dir = g_file_get_child (ostree_dir, "repo");
+  if (!gs_file_ensure_directory (repo_dir, TRUE, cancellable, error))
     goto out;
 
   g_clear_object (&dir);
@@ -223,16 +222,9 @@ ostree_sysroot_ensure_initialized (OstreeSysroot  *self,
   dir = ot_gfile_get_child_build_path (ostree_dir, "repo", "objects", NULL);
   if (!g_file_query_exists (dir, NULL))
     {
-      gs_free char *opt_repo_arg = g_strdup_printf ("--repo=%s/repo",
-                                                      gs_file_get_path_cached (ostree_dir));
-
-      if (!gs_subprocess_simple_run_sync (NULL, GS_SUBPROCESS_STREAM_DISPOSITION_NULL,
-                                          cancellable, error,
-                                          "ostree", opt_repo_arg, "init", NULL))
-        {
-          g_prefix_error (error, "Failed to initialize repository: ");
-          goto out;
-        }
+      gs_unref_object OstreeRepo *repo = ostree_repo_new (repo_dir);
+      if (!ostree_repo_create (repo, OSTREE_REPO_MODE_BARE, cancellable, error))
+        goto out;
     }
 
   ret = TRUE;